Skip to content

HIVE-29631: Remove redundant Maps.fromProperties wrapper#6489

Open
deniskuzZ wants to merge 1 commit into
apache:masterfrom
deniskuzZ:maps_properties
Open

HIVE-29631: Remove redundant Maps.fromProperties wrapper#6489
deniskuzZ wants to merge 1 commit into
apache:masterfrom
deniskuzZ:maps_properties

Conversation

@deniskuzZ
Copy link
Copy Markdown
Member

What changes were proposed in this pull request?

Why are the changes needed?

Does this PR introduce any user-facing change?

How was this patch tested?

Comment thread ql/src/java/org/apache/hadoop/hive/ql/io/AcidUtils.java Outdated
@Aggarwal-Raghav
Copy link
Copy Markdown
Contributor

LGTM +1 (non-binding)

@sonarqubecloud
Copy link
Copy Markdown

Copy link
Copy Markdown
Contributor

@abstractdog abstractdog left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nice refactor, LGTM

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants